Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

README.md: WebStorm/PhpStorm support #345

Merged
merged 2 commits into from
Dec 6, 2015
Merged

README.md: WebStorm/PhpStorm support #345

merged 2 commits into from
Dec 6, 2015

Conversation

jelz
Copy link
Contributor

@jelz jelz commented Nov 25, 2015

This (hopefully) fixes #247

Before merging: dear WebStorm/PhpStorm users, can you try to reproduce my steps and share an outcome here?

@rstacruz
Copy link
Member

@feross, it might be time to have a docs/ folder for standard to refactor the ever-growing README.md.

docs/editor-integration.md
docs/webstorm.md
docs/formatter.md
...

@feross
Copy link
Member

feross commented Dec 2, 2015

These instructions are currently too long to add to the middle of the current readme.

@jelz Can you create a docs/ folder and put these into a webstorm.md file that's linked from the readme?

@jelz
Copy link
Contributor Author

jelz commented Dec 2, 2015

@feross If you wish I can extract all IDE/editor support docs (not only my WebStorm instruction) into separate files.

Either way I'll take care of this PR tomorrow in the evening or during the weekend, depending on my workload.

@feross
Copy link
Member

feross commented Dec 2, 2015

Let's leave the other editor's instructions where they are since they're pretty short. I like that most users can follow the instructions without needing to go to any other pages.

Thanks - looking forward to your updated PR.

@jelz
Copy link
Contributor Author

jelz commented Dec 5, 2015

Updated and rebased onto master.

@feross
Copy link
Member

feross commented Dec 6, 2015

LGTM. Thanks!

feross added a commit that referenced this pull request Dec 6, 2015
README.md: WebStorm/PhpStorm support
@feross feross merged commit 98b376c into standard:master Dec 6, 2015
@lock lock bot locked as resolved and limited conversation to collaborators May 21, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

Webstorm Support
3 participants